Добавление новой способности для SCP-106 "Загрязнение"#1046
Conversation
📝 WalkthroughWalkthroughДобавлена возможность SCP-106 создавать лужи содержащие новый реагент Scp106Reagent: новый реагент, прототип лужи, действие спауна, интеграция в сущность SCP-106 и локализации на en-US и ru-RU. ИзмененияСпособность воспроизведения лужи SCP-106
Диаграмма взаимодействияsequenceDiagram
participant Player as Игрок (SCP-106)
participant Action as Scp106PuddleSpawnAction
participant Spell as InstantSpawnSpellEvent
participant Puddle as PuddleScp106Reagent200
participant Reagent as Scp106Reagent
Player->>Action: Использовать способность
Action->>Spell: Вызвать InstantSpawnSpellEvent (prototype, TargetCasterPos)
Spell->>Puddle: Спаун сущности лужи
Puddle->>Reagent: Решение содержит 200 ед. Scp106Reagent
Reagent->>Player: (при контакте) наносит эффекты: урон, рвота, слепота, замедление
Оценка сложности рецензирования🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (9 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Resources/Locale/ru-RU/_prototypes/_scp/actions/scp106.ftl`:
- Around line 1-2: Переименуйте строку ent-Scp106PuddleSpawnAction так, чтобы
название предмета начиналось со строчной буквы (например "тёмная субстанция") и
исправьте .desc для корректной пунктуации и орфографии (четкая фраза,
запятые/окончания: "Высвобождает среднее количество чёрной гнили, замедляющей
любого, кто наступит на неё." или похожая читабельная формулировка), обновив
соответствующие ключи ent-Scp106PuddleSpawnAction и
ent-Scp106PuddleSpawnAction.desc.
In `@Resources/Prototypes/_Scp/Reagents/scp106.yml`:
- Around line 17-53: Удалить все побочные эффекты, оставив только замедление: в
блоках, где упоминается Scp106Reagent убрать !type:HealthChange, !type:Vomit,
!type:GenericStatusEffect (TemporaryBlindness) и !type:ModifyBleed, а также
полностью удалить или заменить секцию Poison, чтобы она содержала только
!type:MovementSpeedModifier с соответствующими ReagentCondition (min thresholds)
и полями walkSpeedModifier/sprintSpeedModifier; оставьте только элементы с типом
MovementSpeedModifier и связанные ReagentCondition, удалив все остальные записи,
которые наносят урон, рвоту, слепоту или усиливают кровотечение.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 7f17c36c-3f51-47e9-bc5b-ce7fb44d432e
📒 Files selected for processing (8)
Resources/Locale/en-US/_prototypes/_scp/actions/scp106.ftlResources/Locale/en-US/_strings/_scp/reagents/reagents.ftlResources/Locale/ru-RU/_prototypes/_scp/actions/scp106.ftlResources/Locale/ru-RU/_strings/_scp/reagents/reagents.ftlResources/Prototypes/_Scp/Actions/scp106.ymlResources/Prototypes/_Scp/Entities/Effects/puddle.ymlResources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp106.ymlResources/Prototypes/_Scp/Reagents/scp106.yml
|
А так же не |
|
@666XxttimurkaxX666 |
я эту хуйню создал и чо дальше как подписать или что там |
|
a1f08e2 to
1a8325e
Compare
|
да сука всмысле unverified |
1a8325e to
f0230a1
Compare
|
блять что за хуйню он запушил |
|
otkuda ono tut |
|
что за говнооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооо |
|
оно все еще не верифицировано |
f0230a1 to
27c9efe
Compare
|
наконец тооооооооооооооооо |
229f092 to
8d303cf
Compare
8d303cf to
ca55e46
Compare
|
@WardexOfficial а эту фигню с верификацией специально включили для этого проекта или она сама по себе в один момент появилась? |
|
просто у меня из за нее github desktop накрылся и теперь не хочет ничего принимать |
|
@WardexOfficial я починил го мерж |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
Resources/Prototypes/_Scp/Catalog/Scp106Catalog.yml (1)
90-92:⚠️ Potential issue | 🟠 Major | ⚡ Quick winДля
ListingLimitedStockConditionне заданstock.На Line 91-92 условие ограниченного стока объявлено без значения
stock. В лучшем случае лимит не применится как ожидалось, в худшем — конфиг невалиден для этого типа.💡 Предлагаемая правка
conditions: - !type:ListingLimitedStockCondition + stock: 1🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@Resources/Prototypes/_Scp/Catalog/Scp106Catalog.yml` around lines 90 - 92, В блоке conditions у объекта ListingLimitedStockCondition отсутствует обязательное поле stock — откройте определение условия ListingLimitedStockCondition и добавьте ключ stock с целочисленным значением или ссылкой на переменную/параметр (например stock: 10 или stock: ${maxStock}) так чтобы условие ограниченного стока имело корректное число; отредактируйте запись ListingLimitedStockCondition в файле (символ: ListingLimitedStockCondition) чтобы включить stock и сохранить валидность конфигурации.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@Resources/Prototypes/_Scp/Catalog/Scp106Catalog.yml`:
- Around line 90-92: В блоке conditions у объекта ListingLimitedStockCondition
отсутствует обязательное поле stock — откройте определение условия
ListingLimitedStockCondition и добавьте ключ stock с целочисленным значением или
ссылкой на переменную/параметр (например stock: 10 или stock: ${maxStock}) так
чтобы условие ограниченного стока имело корректное число; отредактируйте запись
ListingLimitedStockCondition в файле (символ: ListingLimitedStockCondition)
чтобы включить stock и сохранить валидность конфигурации.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 8bfadf0a-2354-45dc-8f8a-325316cb2e2b
📒 Files selected for processing (11)
Resources/Locale/en-US/_prototypes/_scp/actions/scp106.ftlResources/Locale/en-US/_strings/_scp/catalog/catalog.ftlResources/Locale/en-US/_strings/_scp/reagents/reagents.ftlResources/Locale/ru-RU/_prototypes/_scp/actions/scp106.ftlResources/Locale/ru-RU/_strings/_scp/catalog/catalog.ftlResources/Locale/ru-RU/_strings/_scp/reagents/reagents.ftlResources/Prototypes/_Scp/Actions/scp106.ymlResources/Prototypes/_Scp/Catalog/Scp106Catalog.ymlResources/Prototypes/_Scp/Entities/Effects/puddle.ymlResources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp106.ymlResources/Prototypes/_Scp/Reagents/scp106.yml
|
@WardexOfficial я изменил цену покупки фантома на 0 из за того что люди на 106 могут изначально купить любую другую способность и после не смогут купить фантома (ну то есть это будет софтлок) |
Нужно дать 106 способность становиться фантомом по умолчанию т.к сначала игры можно купить вместо фантома клинок и тоже получится софтлок. |
|
@666XxttimurkaxX666 Да, убери у 106 способность фантома из магазина и добавь её по умолчанию, так же вычти её стоимость из начальных очков магазина 106 |
Краткое описание | Short description
теперь у сцп 106 есть новая способность спавна жижи
если что она только замедляет и всё
Ссылка на багрепорт/Предложение | Related Issue/Bug Report
Медиа (Видео/Скриншоты) | Media (Video/Screenshots)
Changelog
🆑 timur
Summary by CodeRabbit
Новые возможности
Локализация